PR Summary

This PR simplifies the LLM client initialization process by removing the legacy logic used to retrieve an API key from a dynamically constructed ValidMind API endpoint. Specifically, the changes include:

• Eliminating the try-except block in the client setup within the utils module that previously attempted to retrieve an API key and dynamically set the target model and API URL.
• Directly raising a ValueError when the required keys (OPENAI_API_KEY, AZURE_OPENAI_KEY) are not set, removing any fallback behavior.
• Removing the unused function get_ai_key from the api_client module to avoid circular import issues and simplify the overall codebase.

These modifications lead to a cleaner, more maintainable codebase by removing unnecessary complexity and avoiding potential issues with circular imports and dynamic URL handling.

Test Suggestions

• Verify that get_client_and_model raises the appropriate ValueError when the environment is missing the required API keys.
• Test that when valid API keys are set, the correct client (AzureOpenAI or OpenAI) is initialized.
• Ensure that no parts of the codebase reference the removed get_ai_key function, confirming its complete removal.
• Perform integration tests to ensure that the simplified client initialization does not break downstream functionality.